-
Notifications
You must be signed in to change notification settings - Fork 309
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NAS-131424 / 25.04 / Convert more components to standalone #10766
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #10766 +/- ##
==========================================
+ Coverage 80.60% 80.80% +0.20%
==========================================
Files 1570 1571 +1
Lines 51512 52101 +589
Branches 5809 5809
==========================================
+ Hits 41521 42102 +581
- Misses 9991 9999 +8
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
# Conflicts: # src/app/modules/terminal/terminal.module.ts # src/app/pages/account/account.module.ts # src/app/pages/account/groups/groups.module.ts # src/app/pages/account/users/users.module.ts # src/app/pages/api-keys/api-keys.module.ts # src/app/pages/directory-service/directory-service.module.ts # src/app/pages/network/network.module.ts # src/app/pages/services/services.module.ts # src/app/pages/shell/shell.module.ts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
TranslateModule, | ||
TranslateModule, | ||
TranslateModule, | ||
AsyncPipe, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Many duplicates
This PR has been merged and conversations have been locked. |
Testing:
Test some of the affected places.